Skip to content

Conversation

@alex-spies
Copy link
Contributor

@alex-spies alex-spies commented Oct 22, 2025

This will backport the following commits from main to 9.2:

Fix elastic#131509 Fix
elastic#132634

Make `Attribute#equals` and `Alias#equals` respect the `NameId`s, so
that plan transformations that happen to replace an attribute/alias by
one that only differs by id, we still actually update the plan rather
than keeping the old plan object.

This requires a bunch of boilerplate-y test updates. I'll point to the
non-boilerplate changes in comments below.

(cherry picked from commit f7d4422)

# Conflicts:
#	x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/action/EsqlCapabilities.java
#	x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/parser/StatementParserTests.java
@alex-spies alex-spies added :Analytics/ES|QL AKA ESQL >bug auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) labels Oct 22, 2025
@elasticsearchmachine elasticsearchmachine merged commit e6cd8b8 into elastic:9.2 Oct 22, 2025
34 checks passed
@alex-spies alex-spies deleted the backport/9.2/pr-132455 branch October 22, 2025 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport >bug v9.2.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants